Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[강원대 Android_주민철] 2주차 과제 2단계 #65

Open
wants to merge 31 commits into
base: joominchul
Choose a base branch
from

Conversation

joominchul
Copy link

@joominchul joominchul commented Jul 6, 2024

어려웠던 점

  • 1주차 과제에서 피드백 해주신 뷰어댑터에서 클릭 콜백 메서드만 매개 변수로 받는 것, 뷰홀더 이너 클래스 지양하기는 시간이 부족하기도 하고, 아직 감도 제대로 안 잡혀서 적용하지 못 했습니다.
  • MVVM 패턴을 나름 적용한다고 했는데, 제대로 적용이 됐는지 궁금합니다.
  • livedata 개념은 이번에 새롭게 알게 되었는데, 겉핥기 식으로 익혀서 제대로 사용을 했는지 자신이 없습니다.
  • 커밋을 그전에는 자세히 했는데, 이번엔 기능 요구사항 단위로 해보았습니다. 적당한지 궁금합니다.
  • 3단계는 검색어 저장 목록을 리스트뷰로 하는 것인데, 리스트뷰로 가로 스크롤을 어떻게 할 수 있는지 궁금합니다.

joominchul added 20 commits July 5, 2024 11:21
장소 추가 함수
매개 변수로 받은 Place가 존재하는지 확인
추가할 장소가 존재하지 않으면 추가
이름을 검색해서, 검색어를 포함하는 이름을 가진 장소 데이터를 리턴하는 함수
데이터베이스에 데이터가 있는지 확인하는 함수
검색어를 입력하면 검색 결과 목록이 표시되고, 목록은 세로 스크롤이 됨
장소 확인을 like 대신 = 으로 변경
result를 리턴하도록 변경
검색 결과 목록에서 하나의 항목을 선택하면, 검색어 저장 목록에 추가가 되고 가로 스크롤이 됨.
저장된 검색어는 앱을 재실행하여도 유지된다.
Copy link

@mkSpace mkSpace left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

안녕하세요 민철님 예비군도 다녀오시고 학습할 시간이 충분치 않으셨을 텐데도 잘 작성해주셨네요. 이번주도 수고 많으셨습니다. 리뷰 남겨드리니 의견 부탁드려요!

MVVM 아키텍처를 이제 학습해서 바로 적용하기는 쉽지 않죠. 천천히 학습하시고 변경해보시는걸 추천드립니다.

사실 데이터의 흐름이 DB -> View로 흐르게끔 작성하는게 베스트이긴 한데 현재 단계에서 이를 구현하기에는 쉽지 않은 일이죠. 우선 이번 단계에서는 어떤 부분이 문제가 있고 힘든지 힘껏 만끽하셨으면 좋겠습니다.

app/src/main/java/campus/tech/kakao/map/MainActivity.kt Outdated Show resolved Hide resolved
app/src/main/java/campus/tech/kakao/map/MainActivity.kt Outdated Show resolved Hide resolved
app/src/main/java/campus/tech/kakao/map/MainActivity.kt Outdated Show resolved Hide resolved
app/src/main/java/campus/tech/kakao/map/MainViewModel.kt Outdated Show resolved Hide resolved
app/src/main/java/campus/tech/kakao/map/MainViewModel.kt Outdated Show resolved Hide resolved
app/src/main/java/campus/tech/kakao/map/PlaceDbHelper.kt Outdated Show resolved Hide resolved
app/src/main/java/campus/tech/kakao/map/PlaceDbHelper.kt Outdated Show resolved Hide resolved
app/src/main/java/campus/tech/kakao/map/PlaceDbHelper.kt Outdated Show resolved Hide resolved
app/src/main/java/campus/tech/kakao/map/WordAdapter.kt Outdated Show resolved Hide resolved
어댑터들의 클릭 리스너를 메인 액티비티에서 받게끔 변경
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants